-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update pos-consensus.md #298
Conversation
Updating 2x hyperlinks in "Block Production - BABE" section to link to the new "Randomness" page in the Developer Docs (vs the previous links to the equivalent Polkadot Wiki pages).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Just a few minor tweaks
Co-authored-by: Nicolás Hussein <[email protected]>
Co-authored-by: Nicolás Hussein <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Wahoo! Baby's first PR complete 🥳 |
Hey team - just a note, I checked the docs site and it appears the links haven't updated even though this has been approved, looks like something didn't update as expected? @nhussein11 |
Hey @ottovlotto, I’ll check with the team and get back to you |
@ottovlotto, we redeployed the website. Your changes are live: https://docs.polkadot.com/polkadot-protocol/architecture/polkadot-chain/pos-consensus/ Many thanks! |
Updating 2x hyperlinks in "Block Production - BABE" section to link to the new "Randomness" page in the Developer Docs (vs the previous links to the equivalent Polkadot Wiki pages).